Skip to content

Fileset headers#4

Merged
jdiego merged 4 commits intomainfrom
fileset-headers
Jun 22, 2025
Merged

Fileset headers#4
jdiego merged 4 commits intomainfrom
fileset-headers

Conversation

@jdiego
Copy link
Owner

@jdiego jdiego commented Jun 22, 2025

IMPORTANT: Please do not create a Pull Request without creating an issue first.

Any change needs to be discussed before proceeding. Failure to do so may result
in the rejection of the pull request.

Please provide enough information so that others can review your pull request. You
can skip this if you're fixing a typo or adding an app to the Showcase.

Explain the details for making this change. What existing problem does the pull
request solve?

Ex:

  1. If you "Added a/changed the function to do X", explain why:

    • it is necessary to have a way to do X.
    • if there already exists a way, why is your implementation better
  2. If you "Fixed bug/error in X", explain:

    • what was the bug/error (if you already made an issue, please link to it here)
    • how does your implementation fix the issue

Code style and formatting

Check the Contributors Style Guidelines section
for how to write your code and the Contributors Code Formatting section
for how to format your code.

Closing Issues

Put closes #XXXX in your comment to auto-close the issue that your PR fixes
(if such).


Fixes #XXXX

Proposed changes

Motivation behind changes

Test plan

Demonstrate the code is solid. Example: The exact commands you ran and their output,
screenshots / videos if the pull request changes UI.

Make sure tests pass on all of the relevant CI workflows.

Pull Request Readiness Checklist

See details at CONTRIBUTING.md.

  • I agree to contribute to the project under [INSERT PROJECT NAME] (Unlicense)
    License.

  • To the best of my knowledge, the proposed patch is not based on a code under
    GPL or other license that is incompatible with [INSERT PROJECT NAME]

  • The PR is proposed to proper branch

  • There is reference to original bug report and related work

  • There is accuracy test, performance test and test data in the repository,
    if applicable

  • The feature is well documented and sample code can be built with the project
    CMake

@jdiego jdiego self-assigned this Jun 22, 2025
@jdiego jdiego merged commit 16261a3 into main Jun 22, 2025
4 of 5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant